Fix symbolic-icon-translucent-color reftest

When using the pre-rendered png symbolics it seems that we're off a
tiny bit in a few of the pixels on the antialiased borders of a
stroke. To fix this we switch the icon to media-playback-stop-symbolic
which has no such antialiased borders.

I don't quite understand why the pixels are off, this needs more
research.

https://bugzilla.gnome.org/show_bug.cgi?id=734668

Change way alpha is used when recoloring symbolics

If the foreground color has an alpha != 1 we used to just pass that into
the svg. This is useful to e.g. render an insensitive icon. However,
that is not an ideal model for symbolics. For instance, if the icon uses
overlapping areas when drawing, expecting these to be opaque then the
transparent color will result in a different alpha value for the overlapping
area. Also, non-foreground symbolic colors are still rendered opaque, and
the recoloring of pngs can't handle transparent colors.

So, instead we extract any alpha from the foreground, render using the
opaque colors and then apply the foreground alpha to the entire result.
This means we get an even transparency, even for other colors, and we
can apply alpha for the pngs too.

https://bugzilla.gnome.org/show_bug.cgi?id=734668

GtkIconHelper: Short circuit _gtk_icon_helper_get_size in some cases

_gtk_icon_helper_get_size() is often used during size request and may
not necessary mean that the icon will be displayed immediately. In
many common cases we know the size without having to ensure a surface.
In many cases this means we can avoid loading an icon until needed, and
in the case of stateless IconHelpers such as GtkCellRendererPixbuf this
is very important as otherwise it will constantly be reloading icons
if the displayed set is larger than the in-memory icon cache.

https://bugzilla.gnome.org/show_bug.cgi?id=734651

stack: Ensure the bin window has an evmask suitable to windowless children

If a child has set_has_window == FALSE, it purely relies on the events set on
the parent window, for which the bin window used to just ensure the exposure
mask, eating all input events.

https://bugzilla.gnome.org/show_bug.cgi?id=734357

widget: Include event controllers' event mask on gtk_widget_get_events()

gtk_widget_get_events() must indeed tell about events enabled purely through
a GtkEventController, those events will most surely trigger event handlers.

https://bugzilla.gnome.org/show_bug.cgi?id=734357

GtkSpinButton: Only notify 'adjustment' if the property changed

https://bugzilla.gnome.org/show_bug.cgi?id=734649

widget-factory: Add a reset button to the notification

Change the initial value to something that is not a multiple of 3, so
that dismissing the notification on reset makes sense. Let's use 50
because that is already being used elsewhere.

https://bugzilla.gnome.org/show_bug.cgi?id=734614

widget-factory: Turn off focus-on-click for notification close button

... because that is what most applications are doing. So lets do the
same thing for the sake of consistency and setting a good example.

https://bugzilla.gnome.org/show_bug.cgi?id=734614

Reconstruct a missing file

Sadly I had forgotten to add selectionmode.ui to git.
Here is a reconstructed version that seems to work.

treeview: Check the selected node active flag before maybe starting rubberband

This check used to be present in the pre-gestures code, but was unintentionally
removed when splitting code into drag/multiclick gestures. The policy used to
be that if clicking happened on an already selected node, DnD would happen
instead of rubberband selection, so this behavior is resuscitated.

https://bugzilla.gnome.org/show_bug.cgi?id=734143

Bump version

3.13.6

Fix reftests for builddir != srcdir

Make gtk-reftest consult the REFTEST_MODULE_DIR environment
variable to find out where to look for modules, and fix the
libtool hack to construct the .libs subdirectory correctly.

Update DISTCHECK_CONFIGURE_ARGS

The --disable-rebuilds option does not exist anymore, so remove
it. At the same time, add --enable-installed-tests.

Fix animation-direction reftest

This was another victim of clipping changes - the labels were
overdrawing each other, leading to test failure. Prevent this
by separating the grid columns.

wayland: Remove overly complex cursor cache

GtkToolbar: Set clip

This allows outset box shadows to work on toolbars.

https://bugzilla.gnome.org/show_bug.cgi?id=734526

reftests: Make textview-margins pass in make check

I found that setting margins to zero makes the textview clip
away some overshooting pixels. So, instead of a 0/10 split,
do this test with a 2/8 split of margins, to avoid the clipping
issue.

Skip focus tests in make check

These require a window manager to work, and we currently don't
run one when we run make check tests under Xvfb.

Avoid "pkg-config --atleast-version"

AM_PATH_GTK_3_0([3.4], [], [], [gthread]) results in calling
"pkg-config --atleast-version 3.4 gtk+-3.0 gthread-2.0".

With pkg-config 0.28, that test fails, because the version of
gthread-2.0 is smaller than 3.4. This appears to be a behaviour
change since 0.26, but matches what the documentation said in 0.26.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=641638
Bug-Debian: https://bugs.debian.org/756476
Reviewed-by: Matthias Clasen <mclasen@redhat.com>

Updates for 3.13.6

Trivial: Use g_clear_object more

It no longer has atomic overhead, so we can use it everywhere.

Adwaita: GtkSwitch size reduction and tweaks

See https://bugzilla.gnome.org/show_bug.cgi?id=734485

Only show title button separator when there is something to separate

https://bugzilla.gnome.org/show_bug.cgi?id=729636

Revert "Adwaita: comment out the header-bar separator"

This reverts commit b4e7ada6ac52b9930ba8d14a5f4d5d3a009cf163.

Make focus tests session-exclusive

Otherwise, tests running in parallel will steal the focus that
we are trying to test.

Add a selection mode testcase

This shows what one currently has to do manually to get the typical
selection mode with blue headerbar and checkboxes in list rows.
The goal is to make this easier.

Adwaita: pushed button shadows tweakings

Handle ERROR_SXS_PROCESS_DEFAULT_ALREADY_SET

https://bugzilla.gnome.org/show_bug.cgi?id=733773

MSVC Build Files: Use Real GTK+ Version

Make the Visual C++-related build files contain the actual GTK+ version, by
generating them during the configure stage and dist'ing them in the release
tarballs.  This is especially important for builds of introspection files,
as one may need to look at the release version of GTK+ in those files.

MSVC Builds: Apply Visual Style for Win32 Print Dialog

Define ISOLATION_AWARE_ENABLED for the build of the GTK DLL so that visual
style can be applied to the Windows print dialog for all applications using
gtkprintoperation.

Update the script for the generation of gtk-win32.rc for MSVC to not try to
embed the manifest from it (but embed libgtk3.manifest by including it in
the project files, as we are now doing), and embedding the manifest file is
really not supported in MSVC 2010 and later.

Also fix up formatting in the GTK DLL projects.

https://bugzilla.gnome.org/show_bug.cgi?id=733773

Adwaita: progressbar background in scale highlight

Updated Basque language

Updated Basque language

Add missing include to gtk-win32.rc

Otherwise ISOLATIONAWARE_MANIFEST_RESOURCE_ID and RT_MANIFEST constants
can't be expanded.

https://bugzilla.gnome.org/show_bug.cgi?id=733773

drag: add missing introspection annotations

gtk_gesture_drag_get_start_point and gtk_gesture_drag_get_offset
have out args that need to be annotated.

This commit adds the (out) and (nullable) annotations as appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=734292

Updated Slovenian translation

Adwaita: comment out the header-bar separator

Adwaita: linked logic surgery

Adwaita: linked combos

MSVC Builds: Create libgtk3.manifest

Use the Python script that was used to create gtk-win32.rc from
gtk-win32.rc.in to similarly create libgtk3.manifest.  As the wildcard
character can be used for all architectures, use that.

The libgtk3.manifest file needs to be specified explicitly in the projects,
so that it will be recognized and built into the GTK dll, so update the
project files as well on this part.

https://bugzilla.gnome.org/show_bug.cgi?id=733773

gdkwindow-win32.c: Fix build on Visual C++

Since MSVC is more strict about the types that we try to do a compare, cast
old_ptr as HWND, as we know that it is a pointer...

https://bugzilla.gnome.org/show_bug.cgi?id=733768

gtk/gtkprintoperation-win32.c: Fix Missed Include

We need to include gtkprivate.h for _gtk_load_dll_with_libgtk3_manifest().

Updated Slovenian translation

Make sure native W32 print dialog uses visual styles

For that to happen the libgtk3 is embedded with a manifest that requests
common controls library 6.x, and GTK lazily calls InitCommonControlsEx()
to initialize those. Then this manifest is used to temporarily override
the process activation contest when loading comdlg32 (which contains the
code for the print dialog), ensuring that it too depends on common
controls 6.x, even if the application that uses GTK does not.

https://bugzilla.gnome.org/show_bug.cgi?id=733773

When wrapping a foreign W32 window, don't forget to store its HWND

When that window is later used as a parent for a GDK child, the call to
CreateWindow*() will fail, because its HWND is 0.

https://bugzilla.gnome.org/show_bug.cgi?id=733768

inspector: Add a switch to turn off animations

This may be useful in some places to check that application-specific
animations respect this setting.

Suppress a warning from SetWindowLongPtr()

https://bugzilla.gnome.org/show_bug.cgi?id=726224

GtkSwitch: Initialize offset when beginning animation

handle_x always corresponds to the visible position of the handle,
which is where we want to start the animation. Without this, repeated
keyboard activation will not always animate.

GtkSwitch: Update handle_x in size-allocate

Since we are storing positions here that depend on the allocation,
we need to update them in size-allocate. This fixes incorrect
positioning of the handle if the switch is active initially.

https://bugzilla.gnome.org/show_bug.cgi?id=734213

GtkSwitch: Don't store dest_offset

Instead, calculate it on the spot in the tick callback, and
update handle_x in gtk_switch_set_active, based on the new
active property.

GtkSwitch: End animations on finalize

Handle the unlikely case that a switch gets finalized while its
toggle animation is running.

GtkSwitch: End animation in set_active

End any running toggle animation before setting the set,
otherwise things get confused.

GtkSwitch: Also animate for keyboard activation

gtk-encode-symbolic-svg: Fix build with -Werror=format-security

Updated Greek translation

widget-factory: Make one switch start out active

This nicely shows off
https://bugzilla.gnome.org/show_bug.cgi?id=734213

tests: Drop use of deprecated api

Don't use gtk_tree_view_set_rules_hint anymore.

gtk-demo: Drop use of deprecated api

There were several uses of gtk_tree_view_set_rules_hint here.
Just drop them, the theme was ignoring them anyway.

Add a11y state tests

This adds a new test which can be scripted to trigger various
event and action sequences, and record state changes in the
accessibility layer.

So far, there are a few tests verifying state changes when
focus changes.

Related to https://bugzilla.gnome.org/show_bug.cgi?id=715176

MSVC Builds: Build gtk-encode-symbolic-svg

This utility would likely be useful for Windows builds of GTK+, given the
reasons Alex cited for coming up with this utility[1], and MSVC build
support for librsvg is not available at this time (possible, but not
implemented yet).

[1]: https://bugzilla.gnome.org/show_bug.cgi?id=730450

MSVC Builds: Add Script to Generate gtk-win32.rc

In bug 733773, gtk-win32.rc was removed from the dist in the attempt to
make the print dialog on Windows themed, so this script was added so that
the gtk-win32.rc file can be generated during the Visual Studio builds of
GTK+ from the autotools scripts.  This is also intended for filling in the
arch of the build for MSVC builds for the manifest that is also required
for fixing bug 733773.  Python is used as it is already needed for

This is done as a custom build rule in the Visual Studio projects so that
it is easy to clean and rebuild gtk-win32.rc upon an update.

https://bugzilla.gnome.org/show_bug.cgi?id=733773

Fix the build

Quoting accident. "echo -e" is not a command that the shell knows,
echo -e is.

Fix building quartz backend

These were some api changes introduced in:
d930716
48db841
f6ee007

https://bugzilla.gnome.org/show_bug.cgi?id=732137

listbox: Add missing ::row-selected emissions

Rows are not necessarily selected via select_row_internal(), add
the missing signal emissions there. Also the signal should be emitted
when removing the selection altogether.

https://bugzilla.gnome.org/show_bug.cgi?id=729809

Add missing paramter to gtk-menu-tracker-new.

Replace deprecated gtk_application_add_accelerator

With new gtk_application_set_accels_for_action.

Fix build of resource files on OS-X

OS-X's 'echo' command does not have a -e option, so it was included in the files.

Add gtk_application_get_actions_for_accel()

This counterpart to gtk_application_get_accels_for_action() lets you
find out if a particular accelerator has one or more actions associated
with it. This might be useful from an accelerator editor or plugin
system to prevent the the installation of conflicting accelerators.

https://bugzilla.gnome.org/show_bug.cgi?id=721367

GtkTreeView: Avoid deprecation warnings

gtk/Makefile.am: fix gtk-encode-symbolic-svg build for jhbuild

Currently, jhbuild-ing GTK+ on Ubuntu 14.04 LTS and gcc 4.8.2 errors out with
/usr/bin/ld: encodesymbolic.o: undefined reference to symbol 'g_file_new_for_path'
/opt/gnome/lib/libgio-2.0.so.0: error adding symbols: DSO missing from command line
when trying to build gtk-encode-symbolic-svg. This is because $(GTK_DEP_LIBS) isn't defined in $(gtk_encode_symbolic_svg_LDADD) in gtk/Makefile.am. This patch should fix that.

Thanks to b4n and gregier in irc.gimp.net/#gtk+ for help.

https://bugzilla.gnome.org/show_bug.cgi?id=734201

textview: Avoid relocating adjustments on ::size-allocate while these are animating

An animation may be scheduled while the textview content changed in size, so the resize
queued would just unset the animation and set the adjusments with a current value,
defeating gtk_text_view_scroll_to_iter(). In this case, just avoid the adjustment change,
as there is a target value on the way.

https://bugzilla.gnome.org/show_bug.cgi?id=733406

Updated POTFILES.in

window: Update popover positions on ::size-allocate

Popovers may get relocations optimized away if only x/y changed
in the GtkAllocation. So make sure the toplevel updates popover
positions on all situations.

https://bugzilla.gnome.org/show_bug.cgi?id=729140

Add missing include for setlocale

Bug 729924 - Crash trying to print in OSX

We should only use the PageSize option from the ppd if it
actually exists.

widget-factory: Add a linked combobox example

This came up in https://bugzilla.gnome.org/show_bug.cgi?id=733979

GtkSwitch: implement animation

GtkSwitch: remove unused assignment

Ensure an image-missing icon is available

It is bad if the image that is used as a fallback for missing
images goes missing itself, so include it as a resource. This
way, it will always be available.

Embed manifest into gtk-update-icon-cache.exe

This way it doesn't need to be installed alongside it.

https://bugzilla.gnome.org/show_bug.cgi?id=705054

treeview: Deprecated rules-hint

The rules-hint property has always been a fairly bad application API, as
it set some wrong expectations for the developers; deferring to the
theme makes it impossible to design application reliably, and if this is
a usability setting we should either impose this setting on every theme,
or simply drop it.

Our own default theme does not honour the zebra striping, which makes
this function even more questionable.

In practice, usability studies on zebra striping have demonstrated that
alternating colors on a list it improves readability just as much as
clear ruling between rows, or by visually differentiating the selected
row. Zebra striping improves readability (or, at least, it does not
hinder it) on static displays, like a table on paper or a document; on a
dynamic display, like an application's UI, there are different
strategies that yield similar, if not better, results.

https://bugzilla.gnome.org/show_bug.cgi?id=733312

listboxrowaccessible: Avoid assertion failure

The next call to gtk_list_box_get_selection_mode just expected the
GtkListBoxRow's parent to be a GtkListBox and failed when the row was
added to something other than a GtkListBox.

https://bugzilla.gnome.org/show_bug.cgi?id=733782

Add a man page for gtk-encode-symbolic-svg

icon-theme: Support recolorable .symbolic.png files

If an icon theme has a file called "foo-symbolic.symbolic.png" which
was converted from svg using gtk-encode-symbolic-svg we will read
it in an recolor, allowing symbolic icons without using librsvg.

https://bugzilla.gnome.org/show_bug.cgi?id=730450

Add gtk-encode-symbolic-svg

This utility loads a symbolic svg at a specified size and
renders a png file in a special format that can be recolored
later.

https://bugzilla.gnome.org/show_bug.cgi?id=730450

inspector: Add GtkTreeSelection to the object tree

It is not accessible, otherwise.

window: Only avoid queueing resizes if the popover position didn't change

If the same position is requested on a popover, it should at least ensure
the window is realized and raised, even if no resizes are queued on the
content. Otherwise other widgets being mapped might raise the windows over
the popover's if its original position is unchanged.

https://bugzilla.gnome.org/show_bug.cgi?id=734129

gtkcssshadowvalue: Don't double-apply the alpha to shadows

It turns out that when we were painting the shadows, we painted the them
with the base color once, which contained the alpha, and then blurred it
and used it as a mask for the fill, which has the fill again.

To fix this, always paint the base surface with full alpha. The existing
code applies the blur conditionally sometimes in weird ways, so the code
shuffling fix may not look correct, but be assured it is. If the blur
happens, the new cr we return has the *default* color applied, which is
fully opaque black, which works perfectly against the A8 surface.

The fallback spinner code needs some modification, since it is
intentionally using the alpha to paint the lobes which are "in the past".
Since we shouldn't be hitting this fallback path very often, we use a
temporary group and paint it with paint_with_alpha, even though it is
slow.

Small comment fixups

One factual pointed out in the review in
https://bugzilla.gnome.org/show_bug.cgi?id=734053, plus some
formatting nit picks.

gtkcairoblur: Replace our exponential blur with the box blur from mutter

https://bugzilla.gnome.org/show_bug.cgi?id=734053

Add the new debug option to the docs

All debug options are supposed to be listed in the "Running and
Debugging" section of the docs.

GtkMenuTrackerItem: introduce action debugging

Add some action debugging output for GtkMenuTrackerItem as well...

https://bugzilla.gnome.org/show_bug.cgi?id=733965

GtkActionHelper: add some debugging output

Introduce a new debug category "actions" and write some messages from
GtkActionHelper about if we can find the actions or not.

We will probably soon want to add some similar messages to
GtkMenuTrackerItem.

https://bugzilla.gnome.org/show_bug.cgi?id=733965

Remove a reference to a non-existing function

W32: Don't crash when opacity is set for non-toplevel

X11 backend doesn't, and for good reason - main code body does not check
that the window it sets opacity for is, in fact, toplevel.
Just silently fail to do anything for non-toplevel windows.

https://bugzilla.gnome.org/show_bug.cgi?id=733769

Use g_clear_object() shorthand for unref-and-set-to-null

https://bugzilla.gnome.org/show_bug.cgi?id=733766

W32: Don't unregister parent window of an embedded widget

This leads to an assertion failure, because parent window is never registered
in the first place, widget's own GdkWindow is. But that window is unregistered
in a generic fashion by GtkWidget code, so there's nothing for us to do here.

https://bugzilla.gnome.org/show_bug.cgi?id=733766

Remove generated gtk-win32.rc from EXTRA_DIST

https://bugzilla.gnome.org/show_bug.cgi?id=733773

icon-browser: Redo the icon categorization

Instead of slavishly following the naming spec, group the icons
into categories that are more likely to be useful for application
developers. Based on input from Allan Day and Jakup Steiner.